-
-
Notifications
You must be signed in to change notification settings - Fork 1.8k
feat(updater): clear cached installers after update check #9178
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
base: master
Are you sure you want to change the base?
Conversation
🦋 Changeset detectedLatest commit: 0c4eddc The changes in this PR will be included in the next version bump. This PR includes changesets to release 1 package
Not sure what this means? Click here to learn what changesets are. Click here if you're a maintainer who wants to add another changeset to this PR |
This PR is stale because it has been open for 30 days with no activity. Remove stale label or comment, or this will be closed in 30 days. |
@beyondkmp I'm still hesitant on having automatic cleanup up updates. What if we were to open an API that allowed clearing updates programmatically? That would be solving for this issue while avoiding us having to deal with unintended side effects? |
if there's no available update version , then we clear the pending dir to save the disk size.
Tested and confirmed that it can be deleted successfully. Also, when a new version has been downloaded and is in the pending state, checking for updates again will not delete the new version.
Every app checks for updates on startup, and many apps also have a timer that periodically checks for updates in the background. Therefore, performing the deletion during this time should be very appropriate and safe.